Skip to content

Move extended SRT tags behind a extended_tags configuration flag#491

Merged
palemieux merged 2 commits intomasterfrom
issues/0490-srt-tags
Mar 8, 2026
Merged

Move extended SRT tags behind a extended_tags configuration flag#491
palemieux merged 2 commits intomasterfrom
issues/0490-srt-tags

Conversation

@palemieux
Copy link
Copy Markdown
Contributor

Closes #490

@palemieux
Copy link
Copy Markdown
Contributor Author

@nywhere Your quick review would be appreciated.

The new configuration flag is called extended_tags, e.g., {"srt_reader": {"extended_tags": true}}.

Copy link
Copy Markdown
Contributor

@nywhere nywhere left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@palemieux palemieux merged commit e087230 into master Mar 8, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SRT reader doesn't catch {b}, {u}, {i}

2 participants